Added support for more SMS Message optional params#154
Conversation
|
Since I'm looking to use both Whatsapp and the ability to send a message as MMS despite having no mediaUrl, I'm going to try forking this repository and pulling both this and the other PR into it. If it helps anyone or the maintainer or something, I could make a PR containing both. |
|
@onlime any chance you could be merging this? |
…with TwilioContentTemplateMessage implementation
|
ok, merged. But please @jordanmiguel, next time, before pinging everybody, merge in |
|
@onlime Sorry, I didn't see tests were failing because the pipeline didn't run on the other commits. Thanks!! |
|
It appears this PR caused this issue on my sites now Undefined property: NotificationChannels\Twilio\TwilioSmsMessage::$contentVariables Hope this PR can be reversed. I will bump down for now |
|
@onlime there's still a remaining issue that causes this (even with your fixes), I opened the PR for it. |
5 participants
Hi there,
We need the RiskCheck param urgently as Twilio's fraud detection is causing us some issues.
I figured I'd add the rest of the missing params at the same time, might as well :)